Chore: Update tests for interop testing#1225
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Signed-off-by: Varsha B <vab@redhat.com>
2382a0a to
b6ad143
Compare
| // when running against RC/ released version of gitops | ||
| expected_dexVersion = "v2.45.0" | ||
| expected_redisVersion = "8.2.3" | ||
| expected_redisVersion = "7.2.11" |
There was a problem hiding this comment.
Is it expected that the redis version moves backwards?
There was a problem hiding this comment.
I am not sure why this change was made for 1.21 (assuming we will be updating the redis version for 1.22?), one since CI is passing I am not going to change it. But for interop tests we need the version to match the version used in recent 1.21 z stream which is 7.2.11
What type of PR is this?
/kind failing-test
What does this PR do / why we need it:
Have you updated the necessary documentation?
Which issue(s) this PR fixes:
Fixes #?
Test acceptance criteria:
How to test changes / Special notes to the reviewer: